Rename generic agent hinter to hint_use_agent#311
Merged
amanjaiswal73892 merged 8 commits intogeneric_agent_hinterfrom Oct 27, 2025
Merged
Rename generic agent hinter to hint_use_agent#311amanjaiswal73892 merged 8 commits intogeneric_agent_hinterfrom
amanjaiswal73892 merged 8 commits intogeneric_agent_hinterfrom
Conversation
![korbit-ai[bot]](https://avatars.githubusercontent.com/in/322216?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review by Korbit AI
Korbit automatically attempts to detect when you fix issues in new commits.
| Category | Issue | Status |
|---|---|---|
 |
Deprecation notice not logged ▹ view | |
 |
Encapsulate Module Deprecation Logic ▹ view |
Files scanned
| File Path | Reviewed |
|---|---|
| src/agentlab/agents/generic_agent_hinter/init.py | ✅ |
| src/agentlab/agents/hint_use_agent/init.py | ✅ |
Explore our documentation to understand the languages and file types we support and the files we ignore.
Check out our docs on how you can make Korbit work best for you and your team.
Comment on lines
+7
to
11
| warnings.warn( | ||
| f"{OLD} is renamed to {NEW}. {OLD} will be removed in future", | ||
| DeprecationWarning, | ||
| stacklevel=2, | ||
| ) |
There was a problem hiding this comment.
Deprecation notice not logged 
Tell me more
What is the issue?
The code uses a warning to log the deprecation notice, which may not be captured in standard logging systems.
Why this matters
Warnings might be suppressed or not visible in production environments, potentially leading to missed deprecation notices.
Suggested change ∙ Feature Preview
Add a logging statement in addition to the warning:
import logging
logging.warning(f"{OLD} is renamed to {NEW}. {OLD} will be removed in future")
warnings.warn(
f"{OLD} is renamed to {NEW}. {OLD} will be removed in future",
DeprecationWarning,
stacklevel=2,
)Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
Comment on lines
+13
to
+19
| # Alias the top-level | ||
| new_mod = importlib.import_module(NEW) | ||
| sys.modules[OLD] = new_mod | ||
|
|
||
| # Alias known submodules | ||
| for sub in SUBS: | ||
| sys.modules[f"{OLD}.{sub}"] = importlib.import_module(f"{NEW}.{sub}") |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
amanjaiswal73892
added a commit
that referenced
this pull request
Nov 27, 2025
* fixes * add new deps * use external embedding service in task hints retrieval * gpt5 fixes * first cut * update * add event listeners and launcher * Add codegen step-wise recoder agent * adding task hints to generic agent * fix repeated llm configs * load env vars in codegen agent * same hints retrieval for both generic and tooluse agents * filter out current task hints if needed * fix llm config, add gpt-5 * fix * pass new flag and fix db path passing issue * fix goal text * fix current task hints exclusion * remove old reqs * remove recorder from that brach * log task errors * expore agentlabxray * remove commented old chunk * share xray only when env flag present * Add StepWiseQueriesPrompt for enhanced query handling in GenericAgent * update hinting agent retrieval * stepwise hint retrieval * added shrink method * (wip) refactor hinting index * (wip) clean up prompt file * add scripts to run generic and hinter agents, update tmlr config for hinter * move HintsSource to separate hinting file * update hinter agent and prompt * fix prompt for task hint * undo changes to tmlr config * update hinter agent * formatting * bug fix hint retrieval * improve launch script * get queries only for step level hint * Add webarenalite to agentlab loop.py * update stepwise hint queries prompt * fix exc logging * non empty instruction * allow less then max hint queries * add generic agent gpt5-nano config * make ray available on toolkit * check that hints db exists * Fix assignment of queries_for_hints variable * Improve generic agent hinter (#309) * Make LLM retreival topic index selection more robust * add new flag to skip hints with the current goal in the hint source t… (#310) * add new flag to skip hints with the current goal in the hint source traces * Rename generic agent hinter to hint_use_agent (#311) * rename generic_agent_hinter to hint_use_agent for clarity * Add deprecation warning and module alias for generic_agent_hinter * improve module aliasing for submodules * Add todo rename agent name * black * bugfix: check for hint_db only when use_task_hint is true. * fix: address missing initialization and correct args reference in choose_hints method * black * bugfix: skip HintSource init if use_task_hint is false * Fix incorrect references for docs retrieval hinter agent (#313) * address comments * format * Add Environment Variable for Ray port (#315) * add env variable for ray port * document env variables * undo removed llm_config * undo unnessary change * add missing default values for hint prompt flags * black * update names in scripts * use default prompt in hintSource for Tool Use agent * remove experiment scripts --------- Co-authored-by: Oleh Shliazhko <oleh.shliazhko@servicenow.com> Co-authored-by: Hadi Nekoei <had.nekoeiqachkanloo@servicenow.com> Co-authored-by: Oleh Shliazhko <ollmer@users.noreply.github.com> Co-authored-by: recursix <alex.lacoste.shmu@gmail.com> Co-authored-by: Patrice Bechard <patrice.bechard@servicenow.com> Co-authored-by: Hadi Nekoei <hadinekoei94@gmail.com> Co-authored-by: Patrice Bechard <bechardpatrice@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request renamed the agentlab.agents.generic_agent_hinter
module to a new module namedagentlab.agents.hint_use_agent`, and sets up a deprecation warning and import redirection for backward compatibility. The main logic and exports now reside in the new module, while the old module serves as a compatibility layer that warns users and forwards imports.Module rename and automatic forwarding imports:
agentlab.agents.generic_agent_hinterto the newagentlab.agents.hint_use_agentmodule.agentlab.agents.generic_agent_hinterto inform users of the module rename and future removal.agentlab.agents.generic_agent_hinterto dynamically import and alias the new module and its submodules, ensuring existing code that imports from the old module continues to work.Description by Korbit AI
What change is being made?
Rename the generic_agent_hinter package to hint_use_agent, add a deprecation warning for the old name, and alias the old module path to the new one while preserving submodule imports and compatibility.
Why are these changes being made?
To align naming with the new hint_use_agent concept and provide a smooth migration path by aliasing the old package name and warning users about the deprecation.